Skip to content

Require gh dependency visibility for PR automation safety#53

Merged
NagyVikt merged 3 commits intomainfrom
agent/codex/pia-edix-hu-dup-2-add-gh-cli-dependency
Apr 12, 2026
Merged

Require gh dependency visibility for PR automation safety#53
NagyVikt merged 3 commits intomainfrom
agent/codex/pia-edix-hu-dup-2-add-gh-cli-dependency

Conversation

@NagyVikt
Copy link
Copy Markdown
Collaborator

Automated by scripts/agent-branch-finish.sh (PR flow).

NagyVikt and others added 3 commits April 12, 2026 02:58
GuardeX PR/merge workflows rely on GitHub CLI, but setup/status only tracked npm-managed tools. This change adds an explicit required-system-tool check for gh, keeps npm global installs limited to npm packages, and surfaces clear install guidance when gh is missing.

Constraint: gh is a system CLI dependency and cannot be installed via npm package detection

Rejected: Add gh to npm global package list | Would route users to incorrect install path

Confidence: high

Scope-risk: narrow

Directive: Keep gh checks advisory unless workflow policy explicitly requires hard-fail behavior

Tested: npm test; node --check bin/multiagent-safety.js; npm pack --dry-run; git diff --check

Not-tested: OS-specific gh installation commands and auth flows
@NagyVikt NagyVikt merged commit be797ce into main Apr 12, 2026
7 checks passed
@NagyVikt NagyVikt deleted the agent/codex/pia-edix-hu-dup-2-add-gh-cli-dependency branch April 12, 2026 01:15
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant